add Radius testnet / mainnet#3
Conversation
|
You successfully submitted a PR! Due to the amount of PRs coming in: we will only look at PRs that the CI is happy with. We can also not hold your hand getting the CI green - just look how others that where merged did it and RTFM. So as long as there is any CI check that reports an error - no human will look at this. You might be able to ask for some support after supporting the project - e.g. by sending funds to lists.eth. When you fixed things after a requested change - then you also need to (re-)request a review. |
| "name": "Radius Network Testnet", | ||
| "chain": "RAD", | ||
| "name": "Radius Testnet", | ||
| "chain": "RADIUS", |
There was a problem hiding this comment.
I think "Network" is intentional here (cc: @AlBorison @kevinkarwaski )
There was a problem hiding this comment.
I hope we don't do this... Radius Network Testnet sounds clunky.
There was a problem hiding this comment.
I think this is to deduplicate from https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-863.json
There was a problem hiding this comment.
Aah, I see. Do we have to be unique though?
| }, | ||
| "infoURL": "https://radiustech.xyz", | ||
| "shortName": "rad", | ||
| "shortName": "radius-testnet", |
There was a problem hiding this comment.
How long should a shortName be?
| "chainId": 72344, | ||
| "networkId": 72344, | ||
| "icon": "rad", | ||
| "slip44": 1, |
There was a problem hiding this comment.
its a hint for the path of HD wallets - like 1/0/0/0 or whatever. its one of those fields. testnets are always 1 whereas mainnets are like 44 or something - i forget
| "name": "Radius", | ||
| "chain": "RADIUS", |
There was a problem hiding this comment.
This probably also wants to be Radius Network
2 participants
this builds on what @maurermi / @kevinkarwaski did but also adds mainnet. only potentially controversial thing here is not calling it "Radius Network", but rather just "Radius". To me, "Radius Network Testnet" sounds overly wordy.